Skip to content

chore: Upgrade dependencies#93

Merged
spydon merged 5 commits into
flame-engine:mainfrom
CodeDoctorDE:chore/upgrade-dependencies
Jul 8, 2026
Merged

chore: Upgrade dependencies#93
spydon merged 5 commits into
flame-engine:mainfrom
CodeDoctorDE:chore/upgrade-dependencies

Conversation

@CodeDoctorDE

@CodeDoctorDE CodeDoctorDE commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Description

Upgrades the project dependencies and migrates the workspace configuration for the latest Melos setup.
Changes include:

  • Upgrade Melos to ^8.0.0
  • Upgrade xml to ^7.0.0
  • Upgrade dartdoc to ^9.0.0
  • Move Melos configuration from melos.yaml into the root pubspec.yaml
  • Add Dart pub workspace configuration
  • Add resolution: workspace for the tiled package
  • Remove redundant switch break statements reported by the updated analyzer rules

(This is part 1 of upgrading dependencies, if this is published, i can work on the flame.dart upgrades)

Checklist

  • The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
  • I have read the Contributor Guide and followed the process outlined for submitting PRs.
  • I have updated/added tests for ALL new/updated/fixed functionality. (since i don't have new functionality, i don't have tests)
  • I have updated/added relevant documentation in docs and added dartdoc comments with ///. (also)
  • I have updated/added relevant examples in examples. (also)

Breaking Change

  • Yes, this is a breaking change.
  • No, this is not a breaking change.

Related Issues

@spydon

spydon commented Jul 6, 2026

Copy link
Copy Markdown
Member

This PRs description doesn't match with what is in the body. Do note that we shouldn't bump the Dart SDK higher than the Flame min SDK for Dart.

@CodeDoctorDE

CodeDoctorDE commented Jul 6, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

What do you mean it does not match? I updated the version constraints to the newest version.
For xml you need a min. dart version of 3.11 see https://github.com/renggli/dart-xml/blob/a3c1d3fe384c88929a951112ca17b04ee73befbf/pubspec.yaml#L17

With the version bump, the formatter has also changed (see the first pr for the changes without the formatting)

@spydon

spydon commented Jul 6, 2026

Copy link
Copy Markdown
Member

What do you mean it does not match? I updated the version constraints to the newest version. For xml you need a min. dart version of 3.11 see https://github.com/renggli/dart-xml/blob/a3c1d3fe384c88929a951112ca17b04ee73befbf/pubspec.yaml#L17

There are no pubspec updates at all in this PR?

@CodeDoctorDE

CodeDoctorDE commented Jul 6, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author
image

Do you have the same pr open? I also updated melos to have the new syntax inside the pubspec

@spydon

spydon commented Jul 6, 2026

Copy link
Copy Markdown
Member
image

Do you have the same pr open?

Now it's visibly, just GitHub being buggy

spydon
spydon reviewed Jul 6, 2026
View reviewed changes
Comment thread pubspec.yaml Outdated
Comment thread pubspec.yaml Outdated
Comment thread pubspec.yaml Outdated
@CodeDoctorDE

CodeDoctorDE commented Jul 6, 2026

Copy link
Copy Markdown
Contributor Author

Fixed, also updated the github actions to use the new command

@CodeDoctorDE CodeDoctorDE requested a review from spydon July 7, 2026 11:23
spydon
spydon reviewed Jul 8, 2026
View reviewed changes
Comment thread .github/workflows/cicd.yaml Outdated
Comment thread .github/workflows/cicd.yaml
Comment thread pubspec.yaml Outdated
spydon
spydon reviewed Jul 8, 2026
View reviewed changes
Comment thread .github/workflows/cicd.yaml Outdated
@spydon spydon enabled auto-merge (squash) July 8, 2026 12:43
@spydon spydon merged commit 480e718 into flame-engine:main Jul 8, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@spydon spydon spydon approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@CodeDoctorDE @spydon